Skip to content

Use pragma instead of aspect for Interrupt_Priority on Serial_Port#467

Merged
jklmnn merged 2 commits intomasterfrom
issue-466
Apr 9, 2026
Merged

Use pragma instead of aspect for Interrupt_Priority on Serial_Port#467
jklmnn merged 2 commits intomasterfrom
issue-466

Conversation

@jklmnn
Copy link
Copy Markdown
Member

@jklmnn jklmnn commented Apr 8, 2026

The definition of Serial_Port uses the aspect Interrupt_Priority taking a value from the specified discriminants. However the discriminant is not visible in this scope. Use a pragma in the definition of the type instead where the discriminant is visible.

ref #466

@jklmnn jklmnn self-assigned this Apr 8, 2026
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 8, 2026

CLA assistant check
All committers have signed the CLA.

@jklmnn jklmnn force-pushed the issue-466 branch 4 times, most recently from 0349420 to d6e3d70 Compare April 8, 2026 14:42
@pat-rogers
Copy link
Copy Markdown

pat-rogers commented Apr 8, 2026 via email

@jklmnn
Copy link
Copy Markdown
Member Author

jklmnn commented Apr 8, 2026

Ah, thanks for the heads-up. I will see if I can reproduce the internally and open an issue for that, or if it can be fixed with a newer FSF GNAT build. If we can't immediately fix it here, I'd still go for the current workaround (and add a comment) to have at least the CI working again so we have some testing for the other PRs.

@pat-rogers
Copy link
Copy Markdown

Agreed.
I've been using GNAT Pro versions so there could indeed still be a problem with the FSF version, although I would have thought that it would have caught up with the Pro verion by now.
Anyway, thanks!

jklmnn added 2 commits April 9, 2026 13:25
The definition of Serial_Port uses the aspect Interrupt_Priority taking
a value from the specified discriminants. However the discriminant is
not visible in this scope. Use a pragma in the definition of the type
instead where the discriminant is visible.

ref #466
The function distutils.spawn.find_executable is deprecated and should be
replaced with shutil.which. Distutils itself is also no longer part of
the standard library as of Python 3.12.

ref #466
Copy link
Copy Markdown

@zakaria1193-adacore zakaria1193-adacore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jklmnn jklmnn merged commit 1b562e7 into master Apr 9, 2026
5 checks passed
@jklmnn
Copy link
Copy Markdown
Member Author

jklmnn commented Apr 9, 2026

@pat-rogers can you rebase your PRs and see if the CI is fixed in them? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants